[pull] main from SigNoz:main#747
Merged
Merged
Conversation
* feat(dashboard-v2): route entry — DashboardPageV2 fetches dashboard, gates loading/error * feat(dashboard-v2): presentational container — compose description & sections layout * feat(dashboard-v2): breadcrumbs + header chrome * feat(dashboard-v2): dashboard description — title, meta, actions, rename * feat(dashboard-v2): read-only panels & sections layout
* chore: add metric for waterfall monitoring Gauge for config limit and counter to count large traces * chore: use traces instead of tracedetail in metric name * chore: try removing high cardinality attributes * chore: use typed argument in logs * chore: add another metric to get idea of trace sizes * chore: change config namespace to trace instead of tracedetail To make it consistant with metric namespaces * chore: add unit to the windowed response count metric * chore: use metrics names as per otel conventions * chore: use constants for metric attributes * refactor: return error on metric creation failure * Revert "refactor: return error on metric creation failure" This reverts commit 091c93e. * chore: panic on metric initiliazation error
…eted (#11270) * fix: the query being updated after deleting the variables * fix: use exact variable string match when removing clauses on delete Passing `true` to removeKeysFromExpression removed the first clause whose value contained any `$`, which corrupted expressions when two variables shared the same filter attribute (e.g. $env and $env_region both backed by deployment.environment). Switching to the exact variable string (`$${variableName}`) ensures only the deleted variable's clause is removed. Also adds 9 targeted edge-case tests covering shared-key variables, variable-name boundary ($env vs $environment), mixed literal/variable clauses, multi-value array filter items, clickhouse_sql, idempotency, empty widgets, and unrelated-variable no-ops. * fix: refined the deletiong process * fix: adding toast * fix: resolved comments * fix: updated the tests and moved func to utils
…11576) Removes the useEffect that captured a Sentry warning when a widget's query range was not called within 120s, along with the now-unused queryRangeCalledRef and the @sentry/react import in GridCard. Closes SigNoz/engineering-pod#5217
* test(alerts): add tests for triggered and list alerts page * test(alerts): add tests for list alerts and triggered alerts * test(fireEvent): replace by userEvent
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )